-
-
Notifications
You must be signed in to change notification settings - Fork 47
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add writeValueWithoutResponse
and writeValueWithResponse
methods
#47
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @derwehr, thanks for this PR. I would like to merge as soon as possibile, but before that, I need that you add some unit tests on these functions.
Can you do that?
Thanks for your contribution
Hi @chrvadala, thanks for the quick reply. I have now added tests. |
Thanks @derwehr, I will merge and publish in a couple days. Thanks a lot |
I released this PR with 1.9.0 and I added your name into contributions list. |
…WithResponse` methods Add type declarations for the methods `writeValueWithoutResponse` and `writeValueWithResponse` that were added to `GattCharacteristic` with chrvadala#47
… `writeValueWithResponse` (chrvadala#63) * add type declarations for `writeValueWithoutResponse` and `writeValueWithResponse` methods Add type declarations for the methods `writeValueWithoutResponse` and `writeValueWithResponse` that were added to `GattCharacteristic` with chrvadala#47 * Fix typo
The WebBluetooth specs deprecated
writeValue()
. So I addedwriteValueWithoutResponse
andwriteValueWithResponse
to be back on par with the spec